overlay: Don't offset the main window snapshot

We need the node without an offset applied.

Fixes the rose picture being offset in tests/testoverlay

testoverlay: Re-enable blurring

Accidentally disabled in 8b14c8d0a820e6617bd72a39a53d2ae6fa7b242b

label: Add missing nullable annotation

Update Polish translation

spinbutton: Use increment/decrement icon names

And add list-{remove,add}-symbolic as fallback.

Fixes #1569

label: Skip updating link state if we have no layout

This can happen whenever the ::activate-link handler sets different
markup on the label, causing all links to be recreated. In this case,
the GtkLabelLink* passed to emit_activate_link is garbage after the
g_signal_emit call and we shouldn't try to do anything with it.

Fixes #1498

snapshot: Get new GArray item directly

Instead of building it up ourselves, then coping it into the GArray and
then getting the last element of that array.

Use the IEC power symbols for GtkSwitch labels

Unicode 9.0 introduced glyps for the "on" and "off" power states, in the
form of:

 - U+23FD POWER ON SYMBOL, or ⏽
 - U+2B58 HEAVY CIRCLE, or ⭘

With `HEAVY CIRCLE` as "power off symbol" selected to avoid adding yet
another circle to the standard.

Since we moved GtkSwitch to always show glyphs instead of (translatable)
strings, asking the localisation teams to either come up with a suitable
short string to replace the English "ON" and "OFF", or to fall back to
Unicode glyphs, we should ensure we're using the appropriate symbols to
begin with.

See also: gtk!503 for the corresponding gtk-3-24 change.

Merge branch 'wip/muktupavels/listbox-master' into 'master'

listbox: set selected_row before emitting signal

See merge request GNOME/gtk!502

listbox: set selected_row before emitting signal

Signal emittion was added in 6f857f87dcd9 commit and it seems that
this is only place where selected_row is set after emitting signal.

Because of this gtk_list_box_get_selected_row currently returns NULL
as selected row if selection mode is set to GTK_SELECTION_BROWSE.

Merge branch 'wip/dboles/EventControllerKey-docs-4' into 'master'

EventControllerKey: Add missing documentation bits

See merge request GNOME/gtk!454

switch: Always use I/O

This allows for smaller switches, and goes better with
modern themes that have round switches.

gl renderer: Dont't try to draw glyphs at scale 0

Fixes #1475

gl glyphcache: Only support one dirty glyph per atlas

testoverlay: Fix fancy effect

iconcache: Remove unused defines

gl renderer: Remove debug code

gl renderer: Fix hidpi transform nodes

shadowvalue: Remove unused API

gl renderer: Only compile the vertex shader once

All our programs use the same vertex shader, so don't compile it over
and over again. This improves startup times by at least 0.001%, I swear.

gl renderer: Don't forget the offset when rendering blur nodes

Update Spanish translation

Merge branch 'patch-1' into 'master'

Fix a typo in “popup-context-menu” documentation.

See merge request GNOME/gtk!500

Fix a typo in “popup-context-menu” documentation.

Merge branch 'wip/sadiq/750-gtk4' into 'master'

gtkpopover: Fix popover when parent widget is partially visible

Closes #750

See merge request GNOME/gtk!496

gtkpopover: Fix popover when parent widget is partially visible

fixes https://gitlab.gnome.org/GNOME/gtk/issues/750

Updated Spanish translation

Merge branch 'meson-warning' into 'master'

build: Avoid a Meson deprecation warning

See merge request GNOME/gtk!493

build: Avoid a Meson deprecation warning

Using an empty `configuration_data` object to copy a configuration file
is deprecated since Meson 0.47 (released July 2018); the equivalent
behaviour is available by using `copy: true`.

revealer: Handle child-visible properly

The target position is irrelevant for determining if the child should be
visible. When the current position is 0, it needs to be hidden, period.

Fixes #1355

settings: Don't special case IM modules

The previous fixes made it unnecessary to hardcode IM modules for
different display types. The code now automatically skips system IM
modules for other displays.

immodule: Check if module matches display

When loading IM modules from environment variables or GtkSettings, we
would accept "xim" on Wayland. That is a bad idea.

immodule: Don't load unsupported IM module

The code would technically allow loading the xim module when X11 support
was not compiled in.

This is probably an artificial concern, because it's pretty hard to
compile XIM support without X11 support, but it makes the code clearer,
so there we go.

immodule: Pass a GdkDisplay for default context ID

After all, the context depends on the display - we want to use the
wayland context for Wayland,  xim for X11 and so on.

immulticontext: Don't have a global_context_id

Context IDs are dependant on the display - both because displays can use
different backends, but also because changing the GtkSetting is a
per-display operation.

So just remove the cache.
If it turns out we need a per-display cache, we can add one to
GtkSettings.

a11y: Check display in `*grab_cell_focus()`

Calling the accessibility function `grab_focus()` on a `GtkCell` under
Wayland will cause the client to crash.

This is another case of `gdk_x11_get_server_time()` being called
regardless of the actual windowing backend used.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1507

flowbox: Accept multipress gesture late

So it's able to operate properly with the DnD gesture set by
gtk_drag_source_set(). We usually just react on button release,
that's the right time to claim the gesture.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1557

searchenginequartz: Fix naming of Beagle @ comment

As funny as that is, it's not very instructive. :P

https://gitlab.gnome.org/GNOME/gtk/merge_requests/490#note_400993

[GtkSearchEngineQuartz] Limit the returned hits.

When the result set is larger than the limit.

[GtkSearchEngineQuartz] Resolve the path to a GtkFile* for the hit list.

Instead of a char* path, which the search_hit parser doesn't expect,
causing a crash.

Closes: $https://gitlab.gnome.org/GNOME/gtk/issues/815

Merge branch 'fix-corner-resize' into 'master'

Fix resize corners

See merge request GNOME/gtk!481

Merge branch 'commit-docs' into 'master'

docs: Move commit style docs to the contribution guide

See merge request GNOME/gtk!358

Update Japanese translation

Update Japanese translation

(cherry picked from commit 69ddfcf1ed165eb9b092f4da8a4b7396290487b0)

Updated Lithuanian translation

imcontextwayland: (Hopefully) fix if expression

Update POTFILES.in

buttonbox: Remove redundant if statement

This used to do something in the past.

colorchooser: Remove extra custom color

popover demo: Fix size-allocate callback signature

window: Fix state_flags_changed impl name

This is unrelated to GdkSurface and was changed by accident.

stack: Don't queue an unnecessary redraw

Both queue_allocate and queue_resize already queue a draw.

entry: Remove unused struct

Check for the session bus before using it

Patch by Chris Allen, #475

Merge branch 'wip/nbenitezl/move-bookmark-placeholder-index-4' into 'master'

gtkplacessidebar.c: move bookmark at the placeholder index

See merge request GNOME/gtk!459

Merge branch 'patch-1' into 'master'

Zero-fill new GtkTextIter

See merge request GNOME/gtk!455

Merge branch 'wip/carlosg/imwayland-for-master' into 'master'

Use zwp_text_input_v3 on gtk4

Closes #1317 and #1525

See merge request GNOME/gtk!462

Fix XChangeProperty calls

They must be long...

Closes: #1556

Fix resize corners

I was stuck in an X session and noticed that my resize corners
all got east or north cursors. It turns out that gnome-shell
does not properly advertise support for edge constraints under X11,
and the absence of that makes the code for determining the edge
under the cursor misbehave.

This change should fix that.

Merge branch 'wip/carlosg/revealer-fix-interrupted-animations-master' into 'master'

revealer: Fully set the target state if unmapped during animation

See merge request GNOME/gtk!466

image: Avoid some unnecessary signal connections

Equivalent if fd9879e5ffba2c58b897155b39ff79584b3618e0 but for GtkImage.

image: Indentation fix

gl renderer: Add a clip stack

So we can check that the currently set clip is the first one and now
intersect with it. This first clip is always the entire viewport or the
entire render_area and we don't want to end up drawing things to a
texture because of it.

picture: Avoid some unnecessary signal connections

We don't need to ever invalidate the picture size if the paintable tells
us its size is static. Same for the contents.

image: Remove private struct from public header

CellAccessible: Fix return type of row|column_span

See:
 * c876c74eb73fe6daa279b42f9e0a9566ba1ced87
 * 63e0eb5de5d5a14afadebab31fe679beb6706cb2

Update Catalan translation

docs: Clarify the section on dependencies

docs: Clarify the section on commit access

The section is about rules for direct commit access to the upstream
repository, so let's be more clear about it.

docs: Clarify autoclosing issues syntax

Link to the GitLab documentation, and clarify that if no single commit
in a merge requests closes an issue, you should add a reference to the
issue in the commit message anyway.

docs: Make the contribution guide slightly more friendly

This is an important document for newcomers, so we should err on the
side of being more detailed on what kind of contributions we expect,
and how we expect them.

The text is heavily modelled on the contributing-template by Nadia
Eghbal available here:

  https://github.com/nayafia/contributing-template

docs: Move commit style docs to the contribution guide

There's no point in having a separate file detailing how commits ought
to work, considering we already have a contribution guide.

Update Catalan translation

Update Catalan translation

Update Catalan translation

Merge branch 'fix-typo-win32-compilation-speedup-macro-gtk4' into 'master'

Win32: Fix typo on compilation speedup macro define [Gtk4]

See merge request GNOME/gtk!473

Annotate values of PRIORITY constants

g-ir-scanner incorrectly evaluates macro definition that include
references to other macro definitions. Provide a correct value as an
annotation.

Differences in generated gir files:

```diff
@@ -19017 +19017 @@
-    <constant name="PRIORITY_REDRAW" value="20" c:type="GDK_PRIORITY_REDRAW">
+    <constant name="PRIORITY_REDRAW" value="120" c:type="GDK_PRIORITY_REDRAW">
@@ -74229,3 +74229,3 @@
     </constant>
-    <constant name="PRIORITY_RESIZE" value="10" c:type="GTK_PRIORITY_RESIZE">
+    <constant name="PRIORITY_RESIZE" value="110" c:type="GTK_PRIORITY_RESIZE">
       <doc xml:space="preserve">Use this priority for functionality related to size allocation.
@@ -106786,3 +106786,3 @@
     <constant name="TEXT_VIEW_PRIORITY_VALIDATE"
-              value="5"
+              value="125"
               c:type="GTK_TEXT_VIEW_PRIORITY_VALIDATE">
```

See !472

Update Catalan translation

Win32: Fix typo on compilation speedup macro define

There is a typo, the correct macro to define is WIN32_LEAN_AND_MEAN.
After this change <shellapi.h> must be included in order to use
ExtractIconExW().

mediastream: Fix possible memory leak

@error is (transfer full).  So the error passed should be freed if not used

revealer: Fully set the target state if unmapped during animation

If the revealer is told do animate and then unrealize itself, we do
(correctly) stop the animation, but used to do a shortcut where we
just set the target state as current.

Other things are dependent on the animation properly finishing though,
like the contained widget child visibility. This may lead to inconsistent
state where gtk_revealer_get_child_revealed() returns TRUE but the child
widget is unmapped, or vice-versa.

Fully finish the animation here, so the child state is coherent the next
time the revealer is mapped. We can also skip notifying on the property
since it will be handled by gtk_revealer_set_position().

x11: make the tool lookup dependent on the hw id as well

Tools on the same physical item have the same serial number, so the eraser
and the pen part of a single pen share that serial number. With the current
lookup code, we'll always return whichever tool comes first into proximity.

Change the code to use the hw id in addition to the serial number, this way we
can differ between two tools.

x11: don't add unknown tools to our list

Generic tools (Bamboo, built-in tablets) always have the same serial number
assigned by the wacom driver. This includes the touch tool when the wacom
driver handles the touch evdev node (common where users require the wacom
gestures to work).

When the first device is the touch device, a tool is created with that serial.
All future tools now return the touch tool on lookup since they all share the
same serial number. Worse, this happens *across* devices, so the pen
event node gets assigned the touch tool because they all have the same serial.

Since we don't actually care about the touch as a tool, let's skip any unknown
tool. This captures pads as well.

x11: get the tool type from the wacom driver properties

Any wacom device currently sets the tool type to UNKNOWN. The wacom driver has
a property that exports the tool type as one of stylus, eraser, cursor, pad or
touch. Only three of those are useful here but that's better than having all
of them as unknown.

build: wrap IM protocol building under have_wayland altogether

We poke variables that might not be defined if the wayland backend
is disabled.

imwayland: Respect maximum length of 4000 Bytes on strings being sent.

Hitting the limit will raise protocol errors.

imwayland: Plug leaks

The various strings (pending/current preedit, surrounding, and commit
buffer) are being leaked in the case of GtkIMContext destruction.

imwayland.c: fix formatting

imwayland: rearrange functions to remove prototypes

imwayland: Handle enter and leave events

Before this patch, imwayland would assume that text-input enter and leave events follow the general (wl_keyboard) focus, and was unable to handle the situation where they would not be provided at the same time.

imwayland: Collect return value from ::delete-surrounding signal

There's not much we can do about the signal not being handled, but
we should fetch the return value anyway.

imwayland: Don't reset serial while text-input is alive

The serial number is a persistent property of the text-input object.

imwayland: Fix clearing of preedit text in webkitgtk

Fixes webkitgtk misbehaviour as outlined in https://gitlab.gnome.org/GNOME/gtk/issues/1316#note_312942 , which was introduced in 49b17e6c.
The preedit will be cleared on exit only if it is already present.

imwayland: Fix clearing of preedit text

Fixes terminal emulator misbehaviour as outlined in https://gitlab.gnome.org/GNOME/gtk/issues/1316, which was introduced in 49b17e6c. The original commit cleared preedit text by setting it to an empty string, which still counted as existing preedit. The fix sets preedit string to null, which is correctly understood as not present.

modules: Check current context before retrieving surrounding

There may be situations where this might get called while the
currently focused context just went away (eg. after setting the
text widget unsensitive).

Closes: #1317

imwayland: clear preedit on focus out

imwayland: Add text-input-unstable-v3 support

The wayland input module now represents text-input-unstable-v3 support, while the old module supporting gtk-text-input was renamed to gtkwayland.

gstmedia: Implement error handling.

I wonder who forgot that.
Whoops.

mediastream: Allow multiple calls to gtk_media_stream_error()

Just ignore all further ones.

a11y: Fix function return type

Typo right there.